Skip to content

Split offstyledb_core.inc into focused include files#28

Merged
dowoge merged 1 commit intomainfrom
refactor-file-split
Apr 19, 2026
Merged

Split offstyledb_core.inc into focused include files#28
dowoge merged 1 commit intomainfrom
refactor-file-split

Conversation

@dowoge
Copy link
Copy Markdown
Member

@dowoge dowoge commented Apr 19, 2026

No behaviour changes. Moves the single ~1460-line include into six smaller ones, each responsible for one area:

  • offstyledb.inc top-level orchestrator: globals, myinfo, AskPluginLoad2, plugin event handlers
  • offstyledb_shavit.inc v3/v4 shavit compat: native/forward stubs, temp replay path helpers, v3 DB stocks
  • offstyledb_http.inc AddHeaders + Crypt_Base64Encode + API_BASE_URL
  • offstyledb_styles.inc style mapping fetch, cache, view/refresh cmds
  • offstyledb_records.inc Shavit_OnFinish/OnReplaySaved, SendRecord and replay upload callbacks
  • offstyledb_bulk.inc bulk verification + SQL scrape + batch pipeline

Prep for the submission-pipeline redesign

No behaviour changes. Moves the single ~1460-line include into six
smaller ones, each responsible for one area:

- offstyledb.inc         top-level orchestrator: globals, myinfo, AskPluginLoad2, plugin event handlers
- offstyledb_shavit.inc  v3/v4 shavit compat: native/forward stubs, temp replay path helpers, v3 DB stocks
- offstyledb_http.inc    AddHeaders + Crypt_Base64Encode + API_BASE_URL
- offstyledb_styles.inc  style mapping fetch, cache, view/refresh cmds
- offstyledb_records.inc Shavit_OnFinish/OnReplaySaved, SendRecord and replay upload callbacks
- offstyledb_bulk.inc    bulk verification + SQL scrape + batch pipeline

Prep for the submission-pipeline redesign
@dowoge dowoge added the enhancement New feature or request label Apr 19, 2026
@dowoge
Copy link
Copy Markdown
Member Author

dowoge commented Apr 19, 2026

it compiled so LGTM!!

@dowoge dowoge merged commit 5207d43 into main Apr 19, 2026
3 checks passed
@dowoge dowoge deleted the refactor-file-split branch April 19, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant